Skip to content

sanitize markdown - #368

Merged
bourgeoa merged 1 commit into
mainfrom
sanitize
Mar 9, 2023
Merged

bourgeoa merged 1 commit into
mainfrom
sanitize

Conversation

@bourgeoa

@bourgeoa bourgeoa commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@bourgeoa

bourgeoa commented Mar 8, 2023

Copy link
Copy Markdown
Contributor Author

There are no tests. Just did :


<script>alert(1)</script>
<iframe src="https://example.com"></iframe>

This is [XSS](javascript:alert(1))

  • result is :
<p>This is <a>XSS</a></p>

I kept marked and dompurify separate for a better understanding if we ever decide to change one or the other package.

@bourgeoa bourgeoa linked an issue Mar 8, 2023 that may be closed by this pull request
@bourgeoa
bourgeoa requested a review from timea-solid March 8, 2023 18:58
@bourgeoa
bourgeoa merged commit 282a8fe into main Mar 9, 2023
@bourgeoa
bourgeoa deleted the sanitize branch March 9, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

XSS in markdown renderer

2 participants